-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Discover] Support for Security Solution flyout #192934
base: main
Are you sure you want to change the base?
[Discover] Support for Security Solution flyout #192934
Conversation
… in useRootProfile, align root profile resolution in surrounding docs and single docs by using useRootProfile, and add getAppWrapper support to surrounding docs and single doc
🤖 Jobs for this PR can be triggered through checkboxes. 🚧
ℹ️ To trigger the CI, please tick the checkbox below 👇
|
💔 Build FailedFailed CI Steps
Test Failures
Metrics [docs]Async chunks
Page load bundle
HistoryTo update your PR or re-run it, just comment with: cc @davismcphee |
); | ||
}; | ||
|
||
export const getRenderDocViewerFlyout: DataSourceProfileProvider['profile']['getRenderDocViewerFlyout'] = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or getDocViewerReplacement
export function DiscoverGridFlyout(props: DiscoverGridFlyoutProps) { | ||
const getRenderDocViewerFlyout = useProfileAccessor('getRenderDocViewerFlyout', { | ||
record: props.hit, | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @davismcphee for creating this extension point. Althought this extension point works to display our flyout. I am still working on that. What it does not do is gives any handle of onClose
event to the users of extension point.
For example, consider 2 use cases:
-
Users click
close
button of the flyout, in that case we want toeffect
the expand icon change in the Discover DataGrid as seen below: -
User clicking on
expand
button toggles the flyout but the extension point does not have access to know when user has clicked expand icon.
I think giving access to setExpandedDoc
should be good enough and may also be more powerful than getRenderDocViewerFlyout
.
What do you think?
profileId: 'example-root-profile', | ||
isExperimental: true, | ||
profile: { | ||
getRenderAppWrapper: (PrevWrapper, params) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far this is working great. Still testing this. Will get back to you.
Summary
WIP.
Checklist
For maintainers